-
Notifications
You must be signed in to change notification settings - Fork 22.5k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add docs for ExecutionWorld of scripting.executeScript API #24316
Conversation
Preview URLs
Flaws (1)Note! 2 documents with no flaws that don't need to be listed. 🎉 URL:
External URLs (1)URL:
(comment last updated: 2023-02-14 13:31:07) |
d487f6b
to
5829ce4
Compare
Related issue: #22409 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@Rob--W I've left some suggestions to correct a few minor points, which I will leave you to apply. Otherwise, it looks good.
files/en-us/mozilla/add-ons/webextensions/api/scripting/executionworld/index.md
Outdated
Show resolved
Hide resolved
files/en-us/mozilla/add-ons/webextensions/api/scripting/executescript/index.md
Outdated
Show resolved
Hide resolved
files/en-us/mozilla/add-ons/webextensions/api/scripting/executionworld/index.md
Outdated
Show resolved
Hide resolved
…ionworld/index.md Co-authored-by: rebloor <[email protected]>
…escript/index.md Co-authored-by: rebloor <[email protected]>
…ionworld/index.md Co-authored-by: rebloor <[email protected]>
Description
This documents the
world
option inscripting.executeScript
(andregisterContentScripts
too, in Chrome).Motivation
This feature is not documented and I saw a request for documentation at #22493.
Additional details
Implemented in https://bugzilla.mozilla.org/show_bug.cgi?id=1759932
Released in Firefox 102 with https://bugzilla.mozilla.org/show_bug.cgi?id=1766615
Related issues and pull requests
Fixes #22493
I added more detail about the environment in #24215